From 48f5df53188382ced355bf8116bbe1b5e1fced08 Mon Sep 17 00:00:00 2001 From: Benjamin Otte Date: Wed, 26 Oct 2016 05:29:07 +0200 Subject: [PATCH] gtk: Add back infrastructure for GTK_DEBUG=updates Includes the ability to turn on updates in the inspector. Animations are now run via a tick function which allows us to neatly overlay a semi-transparent red rectangle and fade it out over time. It also probably enables way more, but somebody with more UI neatness than me needs to figure out what it eanbles first... --- gtk/Makefile.am | 2 + gtk/gtkdebugupdates.c | 318 +++++++++++++++++++++++++++++++++++ gtk/gtkdebugupdatesprivate.h | 40 +++++ gtk/gtkmain.c | 4 + gtk/gtkwindow.c | 12 +- gtk/inspector/visual.c | 9 +- 6 files changed, 379 insertions(+), 6 deletions(-) create mode 100644 gtk/gtkdebugupdates.c create mode 100644 gtk/gtkdebugupdatesprivate.h diff --git a/gtk/Makefile.am b/gtk/Makefile.am index 31108b8fe6..8e17336e25 100644 --- a/gtk/Makefile.am +++ b/gtk/Makefile.am @@ -458,6 +458,7 @@ gtk_private_h_sources = \ gtkcsswin32sizevalueprivate.h \ gtkcsswidgetnodeprivate.h \ gtkcustompaperunixdialog.h \ + gtkdebugupdatesprivate.h \ gtkdialogprivate.h \ gtkdndprivate.h \ gtkentryprivate.h \ @@ -727,6 +728,7 @@ gtk_base_c_sources = \ gtkcssvalue.c \ gtkcsswidgetnode.c \ gtkcsswin32sizevalue.c \ + gtkdebugupdates.c \ gtkdialog.c \ gtkdragsource.c \ gtkdrawingarea.c \ diff --git a/gtk/gtkdebugupdates.c b/gtk/gtkdebugupdates.c new file mode 100644 index 0000000000..1d3ab2133d --- /dev/null +++ b/gtk/gtkdebugupdates.c @@ -0,0 +1,318 @@ +/* GTK - The GIMP Toolkit + * Copyright (C) 2016 Benjamin Otte + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library. If not, see . + */ + +#include "config.h" + +#include "gtkdebugupdatesprivate.h" + +/* duration before we start fading in us */ +#define GDK_DRAW_REGION_MIN_DURATION 50 * 1000 +/* duration when fade is finished in us */ +#define GDK_DRAW_REGION_MAX_DURATION 200 * 1000 + +typedef struct { + gint64 timestamp; + cairo_region_t *region; +} GtkDebugUpdate; + +static gboolean _gtk_debug_updates_enabled = FALSE; + +static GQuark _gtk_debug_updates_quark = 0; + +static void +gtk_debug_updates_init (void) +{ + static gboolean inited = FALSE; + + if (G_LIKELY (inited)) + return; + + _gtk_debug_updates_quark = g_quark_from_static_string ("-gtk-debug-updates"); + + inited = TRUE; +} + +gboolean +gtk_debug_updates_get_enabled (void) +{ + return _gtk_debug_updates_enabled; +} + +void +gtk_debug_updates_set_enabled (gboolean enabled) +{ + if (enabled) + gtk_debug_updates_init (); + + _gtk_debug_updates_enabled = enabled; +} + +gboolean +gtk_debug_updates_get_enabled_for_display (GdkDisplay *display) +{ + if (gtk_debug_updates_get_enabled ()) + return TRUE; + + if (_gtk_debug_updates_quark == 0) + return FALSE; + + return g_object_get_qdata (G_OBJECT (display), _gtk_debug_updates_quark) != NULL; +} + +void +gtk_debug_updates_set_enabled_for_display (GdkDisplay *display, + gboolean enabled) +{ + if (enabled) + { + gtk_debug_updates_init (); + + g_object_set_qdata (G_OBJECT (display), _gtk_debug_updates_quark, GINT_TO_POINTER (TRUE)); + } + else + { + if (_gtk_debug_updates_quark != 0) + g_object_steal_qdata (G_OBJECT (display), _gtk_debug_updates_quark); + } +} + +static void +gtk_debug_update_free (gpointer data) +{ + GtkDebugUpdate *region = data; + + cairo_region_destroy (region->region); + g_slice_free (GtkDebugUpdate, region); +} + +static void +gtk_debug_update_free_queue (gpointer data) +{ + GQueue *queue = data; + + g_queue_free_full (queue, gtk_debug_update_free); +} + +static void +gtk_debug_updates_print (GQueue *queue, + const cairo_region_t *region, + const char *format, + ...) G_GNUC_PRINTF(3,4); +static void +gtk_debug_updates_print (GQueue *queue, + const cairo_region_t *region, + const char *format, + ...) +{ +#if 0 + GString *string = g_string_new (NULL); + va_list args; + + g_string_append_printf (string, "%3u: ", g_queue_get_length (queue)); + + if (region) + { + cairo_rectangle_int_t extents; + + cairo_region_get_extents (region, &extents); + g_string_append_printf (string, "{%u,%u,%u,%u}(%u) ", + extents.x, extents.y, + extents.width, extents.height, + cairo_region_num_rectangles (region)); + } + + va_start (args, format); + g_string_append_vprintf (string, format, args); + va_end (args); + + g_print ("%s\n", string->str); + + g_string_free (string, TRUE); +#endif +} + +static gboolean +gtk_window_manage_updates (GtkWidget *widget, + GdkFrameClock *frame_clock, + gpointer user_data) +{ + GQueue *updates; + GtkDebugUpdate *draw; + cairo_region_t *region; + gint64 timestamp; + GList *l; + + updates = g_object_get_qdata (G_OBJECT (widget), _gtk_debug_updates_quark); + if (updates == NULL) + return FALSE; + timestamp = gdk_frame_clock_get_frame_time (frame_clock); + + gtk_debug_updates_print (updates, NULL, "Managing updates"); + /* First queue an update for all regions. + * While doing so, set the correct timestamp on all untimed regions */ + region = cairo_region_create (); + for (l = g_queue_peek_head_link (updates); l != NULL; l = l->next) + { + draw = l->data; + + if (draw->timestamp == 0) + { + draw->timestamp = timestamp; + gtk_debug_updates_print (updates, draw->region, "Setting timestamp to %lli\n", (long long) timestamp); + } + + cairo_region_union (region, draw->region); + } + gtk_debug_updates_print (updates, region, "Queued update"); + gdk_window_invalidate_region (gtk_widget_get_window (widget), region, TRUE); + cairo_region_destroy (region); + + /* Then remove all outdated regions */ + do { + draw = g_queue_peek_tail (updates); + + if (draw == NULL) + { + gtk_debug_updates_print (updates, NULL, "Empty, no more updates"); + g_object_set_qdata (G_OBJECT (widget), _gtk_debug_updates_quark, NULL); + return FALSE; + } + + if (draw->timestamp + GDK_DRAW_REGION_MAX_DURATION >= timestamp) + break; + + gtk_debug_updates_print (updates, draw->region, "Popped region"); + draw = g_queue_pop_tail (updates); + gtk_debug_update_free (draw); + } while (TRUE); + + return TRUE; +} + +void +gtk_debug_updates_add (GtkWidget *widget, + const cairo_region_t *region) +{ + GQueue *updates; + GtkDebugUpdate *draw; + + if (!gtk_debug_updates_get_enabled_for_display (gtk_widget_get_display (widget))) + return; + + updates = g_object_get_qdata (G_OBJECT (widget), _gtk_debug_updates_quark); + if (updates == NULL) + { + updates = g_queue_new (); + gtk_widget_add_tick_callback (widget, + gtk_window_manage_updates, + NULL, + NULL); + g_object_set_qdata_full (G_OBJECT (widget), + _gtk_debug_updates_quark, + updates, + gtk_debug_update_free_queue); + gtk_debug_updates_print (updates, NULL, "Newly created"); + } + else + { + GtkDebugUpdate *first = g_queue_peek_head (updates); + + if (first->timestamp == 0) + { + cairo_region_union (first->region, region); + gtk_debug_updates_print (updates, first->region, "Added to existing region"); + return; + } + } + + draw = g_slice_new0 (GtkDebugUpdate); + draw->region = cairo_region_copy (region); + g_queue_push_head (updates, draw); + gtk_debug_updates_print (updates, draw->region, "Added new region"); +} + +static void +gtk_debug_updates_queue_get_extents (GQueue *updates, + GdkRectangle *extents) +{ + GtkDebugUpdate *draw; + GdkRectangle rect; + GList *l; + + *extents = (GdkRectangle) { 0, 0, 0, 0 }; + + for (l = g_queue_peek_head_link (updates); l != NULL; l = l->next) + { + draw = l->data; + + cairo_region_get_extents (draw->region, &rect); + gdk_rectangle_union (extents, &rect, extents); + } +} + +GskRenderNode * +gtk_debug_updates_get_render_node (GtkWidget *widget, + GskRenderer *renderer) +{ + GQueue *updates; + GskRenderNode *node; + GtkDebugUpdate *draw; + GdkRectangle rect; + gint64 timestamp; + double progress; + cairo_t *cr; + GList *l; + + if (!gtk_debug_updates_get_enabled_for_display (gtk_widget_get_display (widget))) + return NULL; + + updates = g_object_get_qdata (G_OBJECT (widget), _gtk_debug_updates_quark); + if (updates == NULL) + return NULL; + timestamp = gdk_frame_clock_get_frame_time (gtk_widget_get_frame_clock (widget)); + + gtk_debug_updates_print (updates, NULL, "Painting at %lli", (long long) timestamp); + + node = gsk_renderer_create_render_node (renderer); + gsk_render_node_set_name (node, "Debug Updates"); + gtk_debug_updates_queue_get_extents (updates, &rect); + gsk_render_node_set_bounds (node, &(graphene_rect_t) GRAPHENE_RECT_INIT(rect.x, rect.y, rect.width, rect.height)); + + cr = gsk_render_node_get_draw_context (node); + + for (l = g_queue_peek_head_link (updates); l != NULL; l = l->next) + { + draw = l->data; + if (timestamp - draw->timestamp < GDK_DRAW_REGION_MIN_DURATION) + progress = 0.0; + else if (timestamp - draw->timestamp < GDK_DRAW_REGION_MAX_DURATION) + progress = (double) (timestamp - draw->timestamp - GDK_DRAW_REGION_MIN_DURATION) + / (GDK_DRAW_REGION_MAX_DURATION - GDK_DRAW_REGION_MIN_DURATION); + else + continue; + + cairo_set_source_rgba (cr, 1, 0, 0, 0.4 * (1 - progress)); + gtk_debug_updates_print (updates, draw->region, "Painting with progress %g", progress); + gdk_cairo_region (cr, draw->region); + cairo_fill (cr); + } + + cairo_destroy (cr); + + return node; +} + diff --git a/gtk/gtkdebugupdatesprivate.h b/gtk/gtkdebugupdatesprivate.h new file mode 100644 index 0000000000..713411951c --- /dev/null +++ b/gtk/gtkdebugupdatesprivate.h @@ -0,0 +1,40 @@ +/* GTK - The GIMP Toolkit + * Copyright (C) 2016 Benjamin Otte + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library. If not, see . + */ + +#ifndef __GTK_DEBUG_UPDATES_PRIVATE_H__ +#define __GTK_DEBUG_UPDATES_PRIVATE_H__ + +#include "gtkwidget.h" + +G_BEGIN_DECLS + + +gboolean gtk_debug_updates_get_enabled (void); +void gtk_debug_updates_set_enabled (gboolean enabled); +gboolean gtk_debug_updates_get_enabled_for_display (GdkDisplay *display); +void gtk_debug_updates_set_enabled_for_display (GdkDisplay *display, + gboolean enabled); + +void gtk_debug_updates_add (GtkWidget *widget, + const cairo_region_t *region); +GskRenderNode * gtk_debug_updates_get_render_node (GtkWidget *widget, + GskRenderer *renderer); + + +G_END_DECLS + +#endif /* __GTK_DEBUG_UPDATES_PRIVATE_H__ */ diff --git a/gtk/gtkmain.c b/gtk/gtkmain.c index 854938c936..e7dca63675 100644 --- a/gtk/gtkmain.c +++ b/gtk/gtkmain.c @@ -115,6 +115,7 @@ #include "gtkbox.h" #include "gtkclipboardprivate.h" #include "gtkdebug.h" +#include "gtkdebugupdatesprivate.h" #include "gtkdndprivate.h" #include "gtkmain.h" #include "gtkmenu.h" @@ -714,6 +715,9 @@ do_post_parse_initialization (int *argc, g_log_set_always_fatal (fatal_mask); } + if (debug_flags[0].flags & GTK_DEBUG_UPDATES) + gtk_debug_updates_set_enabled (TRUE); + gtk_widget_set_default_direction (gtk_get_locale_direction ()); _gtk_ensure_resources (); diff --git a/gtk/gtkwindow.c b/gtk/gtkwindow.c index bb4b48a361..81435f515b 100644 --- a/gtk/gtkwindow.c +++ b/gtk/gtkwindow.c @@ -41,6 +41,7 @@ #include "gtkcssiconthemevalueprivate.h" #include "gtkcssrgbavalueprivate.h" #include "gtkcssshadowsvalueprivate.h" +#include "gtkdebugupdatesprivate.h" #include "gtkkeyhash.h" #include "gtkmain.h" #include "gtkmnemonichash.h" @@ -8211,6 +8212,8 @@ gtk_window_queue_draw_region (GtkWidget *widget, return; } + gtk_debug_updates_add (widget, region); + gdk_window_invalidate_region (_gtk_widget_get_window (widget), region, TRUE); } @@ -9412,7 +9415,7 @@ gtk_window_get_render_node (GtkWidget *widget, { GtkWindowPrivate *priv = GTK_WINDOW (widget)->priv; GtkStyleContext *context; - GskRenderNode *node; + GskRenderNode *node, *updates_node; GtkAllocation allocation; GtkBorder window_border; gint title_height; @@ -9506,6 +9509,13 @@ gtk_window_get_render_node (GtkWidget *widget, gtk_container_propagate_render_node (GTK_CONTAINER (widget), renderer, node); + updates_node = gtk_debug_updates_get_render_node (widget, renderer); + if (updates_node) + { + gsk_render_node_append_child (node, updates_node); + gsk_render_node_unref (updates_node); + } + return node; } diff --git a/gtk/inspector/visual.c b/gtk/inspector/visual.c index 3095ce0782..8c44401648 100644 --- a/gtk/inspector/visual.c +++ b/gtk/inspector/visual.c @@ -23,13 +23,14 @@ #include "gtkadjustment.h" #include "gtkbox.h" #include "gtkcomboboxtext.h" +#include "gtkcssproviderprivate.h" +#include "gtkdebugupdatesprivate.h" #include "gtkdebug.h" #include "gtkprivate.h" #include "gtksettings.h" #include "gtkswitch.h" #include "gtkscale.h" #include "gtkwindow.h" -#include "gtkcssproviderprivate.h" #include "fallback-c89.c" @@ -185,13 +186,11 @@ font_scale_entry_activated (GtkEntry *entry, static void updates_activate (GtkSwitch *sw) { -#if 0 gboolean updates; updates = gtk_switch_get_active (sw); - /* FIXME: Do something */ + gtk_debug_updates_set_enabled_for_display (gdk_display_get_default (), updates); redraw_everything (); -#endif } static void @@ -199,7 +198,7 @@ init_updates (GtkInspectorVisual *vis) { gboolean updates = FALSE; - /* FIXME: Do something */ + updates = gtk_debug_updates_get_enabled_for_display (gdk_display_get_default ()); gtk_switch_set_active (GTK_SWITCH (vis->priv->updates_switch), updates); } -- 2.30.2